Skip to content

refactor: re-add the automatic history update after download#593

Merged
arabcoders merged 2 commits into
masterfrom
dev
Apr 26, 2026
Merged

refactor: re-add the automatic history update after download#593
arabcoders merged 2 commits into
masterfrom
dev

Conversation

@arabcoders

@arabcoders arabcoders commented Apr 26, 2026

Copy link
Copy Markdown
Owner

This pull request refactors how history item moves are handled in the application by introducing a reusable handler and improving socket event integration. The main goal is to centralize and streamline the logic for updating the history state when items are moved, ensuring consistency across different components.

History Move Handling Refactor:

  • Introduced a new historyMoveHandler function in useHistoryState.ts to encapsulate the logic for handling history item moves, making it reusable and easier to maintain. This handler updates the history state when an item is moved to history.

  • In history.vue and simple.vue we register the item_moved socket event using the new handler, to handle items update without the need to trigger full API request.

@arabcoders arabcoders merged commit cb1c355 into master Apr 26, 2026
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant